[PE-7250] CoinDetailPage Tests + Integration test reorg #12820
Conversation
|
…jd/coin-page-test-setup # Conflicts: # packages/web/src/components/notification/Notification/UserSubscriptionNotification.test.tsx # packages/web/src/pages/audio-page/WalletModal.tsx # packages/web/src/pages/coin-detail-page/AssetDetailPage.test.tsx # packages/web/src/pages/profile-page/ProfilePage.test.tsx
…jd/coin-page-test-setup
…jd/coin-page-test-setup # Conflicts: # packages/web/src/components/notification/Notification/UserSubscriptionNotification.test.tsx # packages/web/src/pages/coin-detail-page/components/CoinInfoSection.tsx # packages/web/src/pages/profile-page/ProfilePage.test.tsx
| // Removes the long DOM output from failed queries | ||
| configure({ | ||
| getElementError: (message: any) => new Error(message) | ||
| }) |
There was a problem hiding this comment.
on the fence about whether to include this or not but the giant DOM it outputs is pretty useless most of the time since theres so much stuff on screen
There was a problem hiding this comment.
does this drop the section that shows what accessible elements do exist on the page? that part is really useful, the big dom output after is not though
DejayJD
changed the title
| @@ -0,0 +1,199 @@ | |||
| import { userMetadataFromSDK } from '@audius/common/adapters' | |||
| }, | ||
| test: { | ||
| environment: 'jsdom', | ||
| environment: 'happy-dom', |
There was a problem hiding this comment.
cool! so main rationale here is render speed or what?
| ) | ||
| } | ||
|
|
||
| const assertCoinInsightsSection = async () => { |
There was a problem hiding this comment.
nice, liking these helper fns
Description
New tests
New test mock data patterns:
fixtures/folderhappy-dom, experimental for now. Supposed to be faster & lighterHow Has This Been Tested?
vitest